New name for DeferredFutureSingle and DeferredFutureMaybe #1
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@pakko972, j'ai renommé les deux publishers qu'on peut créer avec une closure, et enlevé le namespace
TraitPublishers
qui ne sert plus à rien, du coup:Tu penses que ça peut aider à s'en rappeler ?
On avait aussi
SingleDeferredFuture
etMaybeDeferredFuture
, qui ont l'avantage de commencer par le mot désiré et sont donc sympas avec l'autocompletion. Mais j'ai trouvé queMaybeDeferredFuture
avait trop d'interprétations erronées (genre: "Hu? It may be deferred? What does it mean?").Autres options qui ne m'ont pas convaincu:
AnySinglePublisher.create { ... }
, trop inspiré de RxSwift, pas assez parlant pour les "Combine-natives".AnySinglePublisher.make { ... }
, flouAnySinglePublisher.deferred { ... }
, on s'attend à ce que la closure renvoie un publisher, à cause de Combine.Deferred.AnySinglePublisher.future { ... }
, on s'attend à ce que la closure soit lancée immédiatement, avant la subscription, à cause de Combine.Future (un éjaculateur précoce, ce Combine.Future, ce qui pose plein de problèmes).AnySinglePublisher.deferredFuture { ... }
, plus tellement mieux queDeferredFutureSingle { ... }